Skip to content

deepspeed engine: fp16 support validation on init#4843

Merged
tjruwase merged 2 commits intodeepspeedai:masterfrom
nelyahu:engine_runtime_fp16_validation
Dec 26, 2023
Merged

deepspeed engine: fp16 support validation on init#4843
tjruwase merged 2 commits intodeepspeedai:masterfrom
nelyahu:engine_runtime_fp16_validation

Conversation

@nelyahu
Copy link
Contributor

@nelyahu nelyahu commented Dec 20, 2023

No description provided.

assert not self.elasticity_enabled(), ("Elasticity is not currently supported"
" with model parallelism.")

if self.fp16_enabled() and not get_accelerator().is_fp16_supported():
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please move this into _do_sanity_check()?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tjruwase Done

@nelyahu nelyahu force-pushed the engine_runtime_fp16_validation branch from 2a45ef4 to dd56d55 Compare December 24, 2023 07:01
@tjruwase tjruwase added this pull request to the merge queue Dec 26, 2023
Merged via the queue into deepspeedai:master with commit 3e94f8c Dec 26, 2023
@nelyahu nelyahu deleted the engine_runtime_fp16_validation branch December 28, 2023 12:59
mauryaavinash95 pushed a commit to mauryaavinash95/DeepSpeed that referenced this pull request Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants